Skip to content

[action] [PR:3910] [ZmqOrch] Optimize memory by popping batch size at a time#159

Merged
mssonicbld merged 1 commit intoAzure:202506from
mssonicbld:cherry/msft-202506/3910
Oct 9, 2025
Merged

[action] [PR:3910] [ZmqOrch] Optimize memory by popping batch size at a time#159
mssonicbld merged 1 commit intoAzure:202506from
mssonicbld:cherry/msft-202506/3910

Conversation

@mssonicbld
Copy link
Collaborator

What I did

  1. Used a reference instead of a unnecessary copy of a set object
  2. Optimize memory by popping batch size at a time

NOTE: Please merge only after the below two PR's are merged

sonic-net/sonic-swss-common#1084
sonic-net/sonic-sairedis#1660

Why I did it

To reduce peak memory usage when applying high-volume dash configuration

How I verified it

Details if related

<!--
Please make sure you have read and understood the contribution guildlines:
https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

1. Make sure your commit includes a signature generted with `git commit -s`
2. Make sure your commit title follows the correct format: [component]: description
3. Make sure your commit message contains enough details about the change and related tests
4. Make sure your pull request adds related reviewers, asignees, labels

Please also provide the following information in this pull request:
-->

**What I did**

1. Used a reference instead of a unnecessary copy of a set object
2. Optimize memory by popping batch size at a time

**NOTE: Please merge only after the below two PR's are merged**

sonic-net/sonic-swss-common#1084
sonic-net/sonic-sairedis#1660

**Why I did it**

To reduce peak memory usage when applying high-volume dash configuration

**How I verified it**

**Details if related**
@mssonicbld
Copy link
Collaborator Author

Original PR: sonic-net/sonic-swss#3910

@mssonicbld
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit ff7e8eb into Azure:202506 Oct 9, 2025
5 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant